-
Notifications
You must be signed in to change notification settings - Fork 7
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Implement qir-qsim app for dynamic measurement handling #17
base: main
Are you sure you want to change the base?
Conversation
which was accidentally copied into the top level
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is the tip of the iceberg; we also need to add a unit test.
src/qirqsim/BufferManager.hh
Outdated
|
||
// Define a hash function for std::pair | ||
|
||
struct pair_hash |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
These classes need to be in the qiree
namespace, and the buffer manager in particular should be qiree::detail
in a detail
subdir
Remark: Compares only empty output for now until we find a way to properly store the result and not just print as we go. Temporarily using lowercase qsim. Will modify class names later with the others at the same time.
This commit will fail tests since I am documenting the file name change separately.
bell_ccx.ll to examples folder and dynamicbv.ll to test data folder
The implementation in this PR was authored by @vicenley , @sethrj , and @wongey with contributions from @danclaudino
and @ausbin. This provides a resolution to Issue #3.
Notes:
qsim
libraries. The first build of qiree with this implementation takes a little while...qir-xacc
andqir-qsim
have been defaulted to OFF. They have to be turned ON during cmake for the executable to be generated.group-tuples
option forqsimTuplesRuntime
has been temporarily deactivated. We can reactivate after we are happy with the implementation.--shots
can be triggered as usual.teleport.ll
file was modified slightly to match the one in the paper.@sethrj : For this commit, I had performed a manual comparison with the files from the private repo to the most recent version of QIR-EE and the merge was smooth on my fork. However, I did NOT compare with @ausbin's recent PRs. Hopefully the conflicts will be at a minimum.